Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IS-902 check block number to download snapshot #1784

Merged
merged 7 commits into from
Jan 17, 2024

Conversation

olehnikolaiev
Copy link
Contributor

@olehnikolaiev olehnikolaiev commented Jan 16, 2024

fixes https://github.com/skalenetwork/internal-support/issues/902

blocks downloading of the latest and newest snapshots on the node

tested manually:

  • run skaled with snapshots enabled
  • wait for 3 snapshots to be created
  • send curl request to download the oldest / newest snapshot
  • the download should not start, you should see an error Wrong block number requested

Copy link

codecov bot commented Jan 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d6427e7) 45.70% compared to head (04298fe) 45.73%.
Report is 1 commits behind head on v3.18.0.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           v3.18.0    #1784      +/-   ##
===========================================
+ Coverage    45.70%   45.73%   +0.02%     
===========================================
  Files          358      358              
  Lines        51820    51820              
===========================================
+ Hits         23685    23698      +13     
+ Misses       28135    28122      -13     

dimalit
dimalit previously approved these changes Jan 16, 2024
@olehnikolaiev olehnikolaiev requested a review from dimalit January 17, 2024 18:20
@olehnikolaiev olehnikolaiev merged commit 678f59b into v3.18.0 Jan 17, 2024
8 checks passed
@olehnikolaiev olehnikolaiev deleted the bug/IS-902-allow-uploading-only-one-snapshot branch January 17, 2024 18:23
@github-actions github-actions bot locked and limited conversation to collaborators Jan 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants